Skip to content

Pipe: Fixed the on committed hook square bug & Trimmed the raw tablet hook & Fixed the premature report for source event & Skipped the parsing of time-covered tsFile#17360

Merged
jt2594838 merged 2 commits intomasterfrom
hook
Mar 26, 2026
Merged

Conversation

@Caideyipi
Copy link
Copy Markdown
Collaborator

@Caideyipi Caideyipi commented Mar 25, 2026

Description

As the title said.

This PR has:

  • been self-reviewed.
    • concurrent read
    • concurrent write
    • concurrent read and write
  • added documentation for new or modified features or behaviors.
  • added Javadocs for most classes and all non-trivial methods.
  • added or updated version, license, or notice information
  • added comments explaining the "why" and the intent of the code wherever would not be obvious
    for an unfamiliar reader.
  • added unit tests or modified existing tests to cover new code paths, ensuring the threshold
    for code coverage.
  • added integration tests.
  • been tested in a test IoTDB cluster.
Key changed/added classes (or packages if there are too many classes) in this PR

@Caideyipi Caideyipi changed the title Pipe: Fixed the on committed hook square bug Pipe: Fixed the on committed hook square bug & Trimmed the raw tablet hook & Fixed the premature report for source event Mar 25, 2026
@Caideyipi Caideyipi changed the title Pipe: Fixed the on committed hook square bug & Trimmed the raw tablet hook & Fixed the premature report for source event Pipe: Fixed the on committed hook square bug & Trimmed the raw tablet hook & Fixed the premature report for source event & Skipped the parsing of time-covered tsFile Mar 25, 2026
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Mar 25, 2026

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 25, 2026 •
edited
Loading

Codecov Report

❌ Patch coverage is 25.00000% with 15 lines in your changes missing coverage. Please review.
✅ Project coverage is 39.74%. Comparing base (c8455c1) to head (b2f3528).
⚠️ Report is 2 commits behind head on master.

Files with missing lines Patch % Lines
...ent/common/tablet/PipeRawTabletInsertionEvent.java 15.38% 11 Missing ⚠️
.../event/common/tsfile/PipeTsFileInsertionEvent.java 33.33% 4 Missing ⚠️
Additional details and impacted files 
@@             Coverage Diff              @@
##             master   #17360      +/-   ##
============================================
+ Coverage     39.61%   39.74%   +0.12%     
  Complexity      312      312              
============================================
  Files          5127     5127              
  Lines        346184   346196      +12     
  Branches      44065    44069       +4     
============================================
+ Hits         137135   137581     +446     
+ Misses       209049   208615     -434

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jt2594838 jt2594838 merged commit 6c8e1d5 into master Mar 26, 2026
28 of 30 checks passed
jt2594838 pushed a commit that referenced this pull request Mar 26, 2026
@Caideyipi
[To dev/1.3] Pipe: Fixed the on committed hook square bug & Trimmed t…
81397b2 
…he raw tablet hook & Fixed the premature report for source event & Skipped the parsing of time-covered tsFile (#17360) (#17362)

* new

* fix-data

* fix

* revert

* fix
@jt2594838 jt2594838 deleted the hook branch March 26, 2026 01:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jt2594838 jt2594838 jt2594838 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Caideyipi @jt2594838